Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow Symfony 7.0 #17

Closed
wants to merge 8 commits into from
Closed

Allow Symfony 7.0 #17

wants to merge 8 commits into from

Conversation

starred-gijs
Copy link
Contributor

Tried to match dependencies from https://github.com/facile-it/mongodb-bundle/releases/tag/1.6.0

It needs some work to get all the checks working...

@starred-gijs
Copy link
Contributor Author

Before I spent time on getting these checks fixed, do you intent to support Symfony 7 @Jean85?

@Jean85
Copy link
Member

Jean85 commented Mar 11, 2024

I definitely want to support it! If you can, pleas continue with this, I'm also ok with bumping a couple of dependecies in this way.

Comment on lines 38 to +39
test-composer-install: lock-symfony-3.4 lock-symfony-4.4 lock-symfony-5.0 lock-symfony-6.0
test-composer-install: lock-symfony-5.4 lock-symfony-6.0 lock-symfony-7.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is duplicated, it may not work correctly like this.

Comment on lines +25 to +26
"symfony/framework-bundle": "^5.0 || ^6.0 || ^7.0",
"symfony/messenger": "^5.0 || ^6.0 || ^7.0"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we dropping Symfony 4.4? If we do, we should bump up to 5.4, the other 5.x minors are not LTS.

@@ -11,7 +11,7 @@ services:
depends_on:
- mongo
mongo:
image: mongo:3.4.2
image: mongo:4.4
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we dropping support to Mongo below 4.4?

We need proper changelog details.

Comment on lines 10 to 20
- [
'@Symfony\Component\Serializer\Normalizer\ArrayDenormalizer',
'@Symfony\Component\Serializer\Normalizer\DateTimeNormalizer',
'@Facile\MongoDbMessenger\Tests\End2End\App\SuppressDeprecationNormalizer'
'@Symfony\Component\Serializer\Normalizer\ObjectNormalizer'
]
- ['@Symfony\Component\Serializer\Encoder\JsonEncoder']

Symfony\Component\Serializer\Normalizer\ArrayDenormalizer: ~
Symfony\Component\Serializer\Normalizer\DateTimeNormalizer: ~
Facile\MongoDbMessenger\Tests\End2End\App\SuppressDeprecationNormalizer: ~
Symfony\Component\Serializer\Normalizer\ObjectNormalizer: ~
Symfony\Component\Serializer\Encoder\JsonEncoder: ~
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we dropped the specific normalizer, maybe we can drop the whole service definition? Or it's just temporary to see if we can solve the deprecations?

@starred-gijs
Copy link
Contributor Author

Hi @Jean85, thanks for your comments! Im actually struggling at lot to get the GH Actions to become all green!
So much stuff is out-dated / not supported anymore, and as I'm not the original author of this code, it is a bit of a challenge 😄

My suggestion is to bump Symfony versions to at least 5.4 (but please also consider bumping to 6.0) and drop support for PHP lower than 8.1? What do you think?

@Jean85 Jean85 mentioned this pull request Mar 13, 2024
@Jean85
Copy link
Member

Jean85 commented Mar 13, 2024

@starred-gijs I'm continuing your work in #18 since I hadn't the permission to push here.

I know it's a bit challenging, but I already had to grind my teeth on similar issues in the past, so I know what to do for now. Thanks for the help, I'll try to move it further!

@Jean85 Jean85 closed this Mar 13, 2024
@starred-gijs
Copy link
Contributor Author

Thanks Jean85, your work is definitely appreciated! Let me know if I can help out with anything 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants